Skip to content

fix(plugin): dispatch bp-analyze as Agent from setup to avoid rate limits#188

Merged
onlycastle merged 2 commits intomainfrom
ooo/run/refactor-analysis-dimensions
Mar 26, 2026
Merged

fix(plugin): dispatch bp-analyze as Agent from setup to avoid rate limits#188
onlycastle merged 2 commits intomainfrom
ooo/run/refactor-analysis-dimensions

Conversation

@onlycastle
Copy link
Copy Markdown
Owner

Summary

  • Setup → Analyze flow fix: bp setup now dispatches bp-analyze as an isolated Agent instead of invoking it as an inline skill. This prevents the setup wizard's accumulated context from causing rate limits when analysis starts.
  • Removed misleading option: The "Queue bp analyze for next session" option is replaced with direct execution, since agent-based dispatch already eliminates the rate-limit concern.
  • Includes dimension refactor: This branch also contains the full 5-dimension analysis refactor (AI Partnership, Session Craft, Tool Mastery, Skill Resilience, Session Mastery).

Changes

  • packages/plugin/skills/bp-setup/SKILL.md — Step 6 now dispatches bp-analyze as Agent, removed queue option
  • Full dimension refactor across plugin skills, MCP tools, shared schemas, and tests

Test Plan

  • Clean install plugin from scratch (/plugin marketplace add + /plugin install)
  • Run bp setup → choose "Run bp analyze now" → orchestrator launches in isolated context
  • No rate limit errors at orchestrator startup
  • Full pipeline completes (17 stages)
  • Report generates at localhost:3456

Generated with Claude Code using /ship-it

onlycastle and others added 2 commits March 26, 2026 15:01
…mits

The setup wizard's "Run bp analyze now" option previously invoked
bp-analyze as an inline skill, which inherited the setup session's
accumulated context and hit rate limits. Now dispatches it as an
isolated Agent with a clean context, consistent with how the
orchestrator dispatches its own stages.

Also removes the misleading "Queue for next session" option since
direct execution is now safe.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…c in bp-setup

Address code review feedback: the LLM executing bp-setup had no way
to resolve [PLUGIN_PATH] when dispatching bp-analyze as an Agent.
Added explicit resolution path and restructured the Agent call as a
code block for clarity.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@onlycastle onlycastle force-pushed the ooo/run/refactor-analysis-dimensions branch from da75311 to 33a964b Compare March 26, 2026 22:01
@onlycastle onlycastle merged commit f30f40f into main Mar 26, 2026
@onlycastle onlycastle deleted the ooo/run/refactor-analysis-dimensions branch March 26, 2026 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant